Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: tracks block time using a gauge metric type #1090

Closed
wants to merge 22 commits into from

Conversation

staheri14
Copy link
Contributor

@staheri14 staheri14 commented Sep 21, 2023

No description provided.

dependabot bot and others added 22 commits June 28, 2023 09:53
…1026)

Bumps [github.com/celestiaorg/nmt](https://github.com/celestiaorg/nmt) from 0.16.0 to 0.17.0.
- [Release notes](https://github.com/celestiaorg/nmt/releases)
- [Commits](celestiaorg/nmt@v0.16.0...v0.17.0)

---
updated-dependencies:
- dependency-name: github.com/celestiaorg/nmt
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* feat: pass chain-id in prepareProposal

* chore: proto-gen

---------

Co-authored-by: rachid <chamirachid1@gmail.com>
* feat: add Namespace method to blob

* lint

---------

Co-authored-by: Callum Waters <cmwaters19@gmail.com>
* pass software version from abci to node info

* Update consensus/replay_test.go

Co-authored-by: Rootul P <rootulp@gmail.com>
…1039)

Bumps [github.com/celestiaorg/nmt](https://github.com/celestiaorg/nmt) from 0.17.0 to 0.18.0.
- [Release notes](https://github.com/celestiaorg/nmt/releases)
- [Commits](celestiaorg/nmt@v0.17.0...v0.18.0)

---
updated-dependencies:
- dependency-name: github.com/celestiaorg/nmt
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Evan Forbes <42654277+evan-forbes@users.noreply.github.com>
## Description

Closes #1040

After merging this one, I will cherry-pick this change for main. Then, I
will open a separate PR for main to remove the square size and the data
hash from the `Data` struct.

---


#### PR checklist

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments
## Description

leaving as a draft and WIP until I test this out more. not ready for
review

closes #1004

---------

Co-authored-by: Rootul P <rootulp@gmail.com>
…ensus (#1055)

## Description

This PR updates the trace client with a slight refactor (renaming to be
more consistent) and then adds a few standard tables to store traced
information. We are also able to select which tables we wish to update
(and therefore which information to trace) in the config.

closes #978

---------

Co-authored-by: Rootul P <rootulp@gmail.com>
## Description

since its difficult to test the tracer irl, we missed this silly bug
:facepalm:
## Description

Fixes celestiaorg/orchestrator-relayer#432

#### PR checklist

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments

---------

Co-authored-by: Rootul P <rootulp@gmail.com>
## Description

Fixes: celestiaorg/celestia-app#2249

#### PR checklist

- [ ] Tests written/updated
- [ ] Changelog entry added in `.changelog` (we use
[unclog](https://github.com/informalsystems/unclog) to manage our
changelog)
- [ ] Updated relevant documentation (`docs/` or `spec/`) and code
comments
## Description

To accurately gauge block time, it's essential to track the timestamps
of committed blocks. The modifications made in this PR incorporate this
feature.

Incremental work toward
#1056
## Description

adds more things to the block table, see
#1062 (comment)
for further context.

part of #1056

---------

Co-authored-by: Sanaz Taheri <35961250+staheri14@users.noreply.github.com>
## Description

Adds the last commit median time to `RequestPreparePropsosal`. This
allows the application to accurately calculate the spendable funds of
vesting accounts, since this is the time used when executing the
transactions and when evaluating the propsoal during `ProcessProposal`.

The height was also added, but it doesn't have to be since the
application can calculate that value. That value is also passed in other
versions of comet, and we'll have to add it we want to unfork from the
sdk so I figured why not. Happy to remove it as well.
…otal and message_send_bytes_total metrics (#1078)

Part of #1077
Signed-off-by: dependabot[bot] <support@github.com>
Signed-off-by: Thane Thomson <connect@thanethomson.com>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Co-authored-by: Sergio Mena <sergio@informal.systems>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Jasmina Malicevic <jasmina.dustinac@gmail.com>
Co-authored-by: Lasaro <lasaro@informal.systems>
Co-authored-by: Thane Thomson <connect@thanethomson.com>
Co-authored-by: mmsqe <mavis@crypto.com>
Co-authored-by: yihuang <huang@crypto.com>
Co-authored-by: Steven Ferrer <steven.r.ferrer@gmail.com>
Co-authored-by: Chill Validation <92176880+chillyvee@users.noreply.github.com>
@staheri14 staheri14 self-assigned this Sep 21, 2023
@staheri14 staheri14 closed this Sep 21, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants